axi_to_detailed_mem: Avoid spurious write responses with HideStrb#383
axi_to_detailed_mem: Avoid spurious write responses with HideStrb#383micprog merged 2 commits intopulp-platform:masterfrom
Conversation
|
Could you please check the CI failure? It looks to me like it is unrelated to my changes (failure is while installing Bender). Otherwise, please let me know if you think it's due to my changes. |
|
I'm not sure this PR fixes a symptom or the cause of the actual issue. Could you have a look at #389 to see if that helps fix your issue? |
|
I am not sure if #389 fixes my issue. Also, I have unfortunately not saved that state of the design, so I cannot reproduce the issue anymore. My issue was specific to the case where there is only a single bank, and the full strobe is zero. I know this is kind of weird, but this is what was received in my case (for some reason). I am not well-versed enough in your codebase to quickly write a minimum failing example. Looking at it differently: How do you ensure that there are never more responses (on I don't think #389 actually fixes that issue, but is a different (unrelated) issue. As far as I can tell, the issue fixed by #389 can only occur if there is back-pressure from the memory bank, which was not the case for me (I tied |
|
Ah, I see your point. Thanks for the additional explanation! |
The dead response FIFO could previously underflow, because the surrounding code could present a valid read response (
rvalid_o) based ondead_response, without considering whetherdead_responsewas actually valid.